Skip to content

Conversation

@mvillasante
Copy link
Member

@mvillasante mvillasante commented Oct 3, 2025

Descripción

  • Al trabajar en bycatch_thesis #735 surgió un error al correr esta receta y sus equivalentes.
  • La función get_trips() se quejaba de una columna faltante.
  • Al principio pensamos que se debía a una actualización del seabird.tracking en la función construct_bl_table().
    Sin embargo, vimos que aquí también usamos get_trips().
  • construct_bl_table() debe comer datos crudos con la columna time. La columna time_gmt es un requerimiento de searbird tracking database.
  • La receta de data/processed/trips_geographic_points.csv es redundante, pues bird_life_format_gps_albatross.csv ya tiene etiquetados los viajes.

Commits relevantes

No format supplied for Date and Time fields, a default format ('ymd_HMS')
attempted when combining the fields. If error produced, see help page
('?lubridate::parse_date_time') for information on date formats.

@mvillasante mvillasante requested a review from devarops October 3, 2025 20:44
Copy link
Member

@devarops devarops left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Image

@devarops devarops merged commit 4303a08 into develop Oct 9, 2025
3 checks passed
@devarops devarops deleted the hot_fix/parse_dates branch October 9, 2025 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants